Skip to content

Conversation

@m6z1
Copy link
Member

@m6z1 m6z1 commented Feb 15, 2025

📮 관련 이슈

✍️ 구현 내용

  • 문의하기 링크를 변경된 링크로 수정했습니다.

📷 구현 영상

2025-02-15.7.37.18.mov

✔️ 확인 사항

  • 컨벤션에 맞는 PR 타이틀
  • 관련 이슈 연결
  • PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
  • Github Action 통과

@m6z1 m6z1 added 🤗 FEATURE Develop this project 💛명지 labels Feb 15, 2025
@m6z1 m6z1 requested a review from huiwoo-jo February 15, 2025 10:09
@m6z1 m6z1 self-assigned this Feb 15, 2025
Copy link
Contributor

@huiwoo-jo huiwoo-jo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코멘트만 확인해주세요!

android:allowBackup="true"
android:icon="@mipmap/ic_logo"
android:label="@string/app_name"
android:networkSecurityConfig="@xml/network_security_config"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우리 http를 사용하는 부분이 없나용

Copy link
Member Author

@m6z1 m6z1 Feb 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

있는데 manifest 속성 중 android:usesCleartextTraffic가 이미 true 값으로 설정이 되어있기에 해당 속성은 필요 없습니다. 그리고 network_security_config 파일 자체가 지금 깃에도 올라가있어서 url 이 노출되고 있는 중이라서욥

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아하 그렇군요!
그럼 지워도 괜찮겠네용

@m6z1 m6z1 merged commit c60af32 into develop Feb 17, 2025
1 check passed
@m6z1 m6z1 deleted the feat/change-inquiry-link branch February 17, 2025 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💛명지 🤗 FEATURE Develop this project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

문의하기 링크 수정

3 participants